Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Seatalk1 interface support #2265

Merged
merged 31 commits into from
Jan 28, 2024
Merged

Seatalk1 interface support #2265

merged 31 commits into from
Jan 28, 2024

Conversation

pgrawehr
Copy link
Contributor

@pgrawehr pgrawehr commented Jan 14, 2024 •
edited by ghost
Loading

This adds support for Seatalk1 messages, either via NMEA0183 or directly.

  • A first set of messages is supported, mostly to support older autopilots
  • Schematics for interfacing Seatalk1 with a Raspberry Pi are provided
  • Message translation for NMEA0183<->Seatalk included as well.
Microsoft Reviewers: Open in CodeFlow

@pgrawehr
Build up infrastructure for new binding
0d055da
@pgrawehr
More stuff for first draft
9f5d35e
@pgrawehr
Doesn't compile?
a369e7f
@pgrawehr
First successful parser tests
eefd720
@pgrawehr
It starts to do something useful
baba280
@pgrawehr
More messages, and high-level support
c92423c
@pgrawehr
First messages can be sent
e534f9f
@pgrawehr
It works!
2629339
@pgrawehr
The most important commands work, I can control the AP remotely!
f579d32
@pgrawehr
Deadband mode selection works
4ecbf0c
@pgrawehr
Lots of unit testing
68f2d9e
@pgrawehr
Testing and bugfixing
1d84b44
@pgrawehr
Some new messages added
65badd9
@pgrawehr
Apparent wind command appears to work
15dd3e9 
Can't really verify whether the angle is provided correctly, though
@pgrawehr
Track message and Autopilot Track Mode work
c5b5df5
@pgrawehr
Basic roundtrip for all messages fixed
d5e052a 
Still need to verify individual arguments
@pgrawehr
Add Schematic for interface
bfa3c70
@pgrawehr
Ignore KCad backup directory
83710e2
@pgrawehr
HTD sentence supported
db1237e 
Not yet tested on real system, poorly documented
@pgrawehr
Add support for $STALK message
4164586
@pgrawehr
Raw Seatalk1 to NMEA conversion works
007db2f
@pgrawehr
More simulation stuff
b95310f 
Unfortunately, the ST2000 doesn't contain a rudder sensor,
but at least the data could be sent.
@pgrawehr
Schema cleanup
96c53d0
@pgrawehr
Add some message translation between Seatalk and NMEA
6bb711f
@pgrawehr
Various tests added and NavigationToWaypoint reversed
95339ae
@pgrawehr
Minor cleanup, create empty PCB
9bbff7f
@pgrawehr
Schema updated, PCB drawn (unverified!)
7be59bd
@pgrawehr
Documentation started
38c4225
@pgrawehr
Finalize documentation
353f3d7
@pgrawehr
Fix linter errors
9b04cc9
Ellerbach
Ellerbach approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@Ellerbach Ellerbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very hard to review in details. But as always, great quality. Usage of UnitsNet, code well documented, with tests and docuentation.

@pgrawehr pgrawehr merged commit 5226fe6 into dotnet:main Jan 28, 2024
10 checks passed
@pgrawehr pgrawehr deleted the seatalk1 branch January 28, 2024 19:39
@github-actions github-actions bot locked and limited conversation to collaborators Feb 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Ellerbach Ellerbach Ellerbach approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pgrawehr @Ellerbach